iT邦幫忙

2019 iT 邦幫忙鐵人賽

DAY 30
2

這麼多天以來,看了很多 Laravel 的程式,其實可以發現它有一些常見的模式。沒有好壞,只是一種可參考的寫法:

在 if 裡做 assign

if ($user = $this->resloveUser()) {
    $user->can('create');
}

這樣寫可以省一行定義,但 if 內的事務過於複雜就容易不知道 condition 的副作用為何。

判斷方法存在才呼叫

if (method_exists($middleware, 'handle')) {
    return $middleware->handle();
}

曾有討論是反對這種寫法,因為會覺得直接定介面就好。但反過來說,如果定了介面,就無法改變參數的定義,如果想設計一個可長可短,同時要能自動注入的方法,就是件不可能的任務了:

public function handle($request, $next);
public function handle($request, $next, $arg1);
public function handle($request, $next, $arg1, $arg2);

丟例外有時也會寫方法處理

protected function getRouteForMethods($request, array $methods)
{
    // ...
    $this->methodNotAllowed($methods);
}

protected function methodNotAllowed(array $others)
{
    throw new MethodNotAllowedHttpException($others);
}

常使用 Fluent pattern 來表示一連串的事件流

通常要對同一個物件做操作,且前後操作是有相關聯的時候,使用 fluent pattern 更能感受到程式所想表達的意圖:

Gate::allows('update-post', $post));

Gate::forUser($user)->allows('update-post', $post));

同個方法裡,流程控制區塊盡可能少

流程控制區塊指的是 if / for / foreach / while 等。這些流程都與循環複雜度呈正相關。如果越大,代表這段程式越難懂,也越容易出現 bug。目前有注意最多的地方在 Container,有七個流程控制。

流程控制多採用有問題先處理掉的寫法

比起下面兩種寫法,會是上面比較好懂一點。當流程控制區塊越多的時候,將會更明顯。

if (!$user) {
    return false;
}

if (!is_callable($callback)) {
    return false;
}

return $callback($user);

另一種相反的寫法

if ($user) {
    if (is_callable($callback)) {
        return $callback($user);
    }
}

return false;

後記

筆者時常說:

程式語言也是一種「語言」,跟自然語言有其相像的點。

這次的主題是分析原始碼,同時就有點像在做一個翻譯員,試圖自己了解,也讓各位讀者也能理解程式所想表達的意圖。

有趣的是,這過程就很像在讀小說會感受到文學的美一樣,會覺得程式這樣寫很好理解,甚至很酷。到了自己要寫文章--也就是要寫程式時,就會回頭把酷炫的寫法「借」過來使用,同時也想讓更多人除了使用外,也能了解自己所寫的程式。

而更重要的,程式開發人員,最強大的地方在於,有能力可以理解與分析其他人寫的原始碼。分析原始碼,正是在培養工程師閱讀程式的能力。另外也增加判斷程式好壞能力的好方法,因為唯有同時看過好程式與爛程式,才有可能會知道什麼樣的程式是好的。

這三十天寫的分析,筆者盡可能寫出讓其他開發者能理解的描述,希望能幫助到大家。


上一篇
分析 Auth(6)--Authorization
下一篇
分析自定義錯誤頁
系列文
Laravel 原始碼分析46
圖片
  直播研討會
圖片
{{ item.channelVendor }} {{ item.webinarstarted }} |
{{ formatDate(item.duration) }}
直播中

1 則留言

0
Lucas Yang
iT邦新手 4 級 ‧ 2018-11-26 00:07:18

漏了一個括號

在 if 裡做 assign

                               v 要補一個括號
if ($user = $this->resloveUser() {
    $user->can('create');
}
Miles iT邦新手 2 級 ‧ 2018-11-26 01:09:36 檢舉

真的漏了,感謝提醒

我要留言

立即登入留言